Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Conversation

denis0001-dev
Copy link
Member

No description provided.

alexr00 and others added 30 commits October 6, 2025 16:03
Switches from `node.parent.replaceChild(x, node)` to `node.replaceWith(x)`. This lets us avoid having to check that the parent exists
feat: add 'hiddenAfterComplete' option to tool's presentation prop
* fix for 267145

* fix for 267145: correcting variable name

* Update src/vs/workbench/contrib/debug/browser/repl.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Connor Peet <copeet@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Fix any cast

* Fix some 'as any'
Use `replaceWith` in to avoid not null assertions
Remove `any` casts in some built-in extensions
Fix checking $traceid for empty args
* Git - remove usages of `any`

* 💄 fix check
… args

Part of #269213

This prevents accessing arbitrary properties on the args without first doing a type check or type assertion
…y` for args

Part of #269213

Updates the command service to use `unknown` for args. Does not yet update `ICommandHandler` as this will require many more changes
Update caller side of CommandService to use `unknown` instead of `any` for args
Fix some `any` casts and `any` usage in instantiation service
… Coverage (#269505)

* Add navigation for uncovered lines in test coverage

Implements [#258967](https://github.com/obrobrio2000/vscode/issues/258967)

- Add commands to navigate to next/previous uncovered lines
- Add navigation buttons to coverage toolbar widget
- Add keyboard shortcuts (Alt+F9 / Shift+Alt+F9)
- Navigation wraps around at file boundaries
- Buttons appear in coverage toolbar and editor title menu

This feature helps developers efficiently review coverage gaps
by allowing them to jump between uncovered lines without manual
scrolling through the file.

Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com>

* Update src/vs/workbench/contrib/testing/browser/codeCoverageDecorations.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Fix navigation logic and add sorting for uncovered lines

- Add proper sorting of missed lines by line number
- Fix corrupted control flow in previous/back navigation
- Add missing variable declarations
- Remove duplicate/incorrect conditional blocks

This fixes the navigation to work correctly with properly
ordered missed lines.

Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com>

* feat: add navigation commands for uncovered lines in code coverage

Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com>

* fix: add missing ID property and remove unused method

- Add static ID property to CodeCoverageDecorations class
- Remove unused isMissedLine method (logic handled inline)
- Fixes TypeScript compilation errors

Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com>

* fix: update CodeCoverageDecorations ID to use Testing constant

Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com>

* fix: refactor navigation titles for uncovered lines to use constants

Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com>

---------

Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
mjbvz and others added 30 commits October 9, 2025 09:45
Tries to use the standard markdown actionHandler in more cases

As part of this, switches some callers from the low level `renderMarkdown` function to the `IMarkdownRendererService`
Avoid using custom markdown actionHandlers in a few more cases
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…s picker (#270450)

* Add a button to view extension details from Managed Trusted Extensions picker

* PR feedback
We want to prevent mistaken changes that do something like this:

```ts
foo.onEvent = () => { ... };
```

When they almost always mean:

```ts
foo.onEvent(() => { ... })
```
Co-authored-by: Dmitriy Vasyura <dmitriv@microsoft.com>
Mark most `Event` properties as `readonly`
* Add prompt support to QuickPick

* Update src/vs/platform/quickinput/browser/quickInput.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Fix incorrect suggestion

* PR feedback

* Merge branch 'main' into dev/dmitriv/fix-78335

* PR feedback

* Merge branch 'main' into dev/dmitriv/fix-78335

* Fix build break.
…tion

Fix debug exception navigation appearing in all editor groups
…270476)

* Changes to remove the disabled extensionMCP server from cached servers.

* Adding tests for extensionMCPDiscovery

* removing extra lines

* removing extra lines

* adding more tests
…-JSON responses (#270579)

* Initial plan

* Add Content-Type check before JSON parsing in _getAuthorizationServerMetadata

Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>

* Improve comment explaining Content-Type check fallback behavior

Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>

* Fix: Check both status AND content-type for all metadata URL attempts

Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>

* Refactor: Try parsing JSON payload instead of checking Content-Type

Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>

* Refactor: Use early returns instead of nested if blocks

Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>

* add a log

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com>
Co-authored-by: Tyler Leonhardt <tyleonha@microsoft.com>
* Retry Auth on a 401

If we have an Authorization header and still get a 401 when trying to connect to an MCP Server, we should retry with a new auth registration.

This fix was for the local MCP dev loop, where you may use an in-memory OAuth server. If you kill the server and start it again, any tokens will be rejected. This PR allows us to react to that by re-doing the auth flow.

* Update src/vs/workbench/api/common/extHostMcp.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Update chat code block languages when languages change
Fixed issue with findFiles2 returning dups (#263911 and #263400)
* Changes to remove the disabled extensionMCP server from cached servers.

* Adding tests for extensionMCPDiscovery

* removing extra lines

* removing extra lines

* adding more tests

* Fixing the tests for build error
* Changes to remove the disabled extensionMCP server from cached servers.

* Adding tests for extensionMCPDiscovery

* removing extra lines

* removing extra lines

* adding more tests

* Fixing the tests for build error

* Fixing the tests for build error
Fix auto approve links for new rules
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.